Skip to content

Not everyone uses paru or yay on Arch: Support pacman command#1900

Merged
Purian23 merged 4 commits intoAvengeMedia:masterfrom
merdely:master
Mar 3, 2026
Merged

Not everyone uses paru or yay on Arch: Support pacman command#1900
Purian23 merged 4 commits intoAvengeMedia:masterfrom
merdely:master

Conversation

@merdely
Copy link
Contributor

@merdely merdely commented Mar 3, 2026

Allow the System Update widget to work on Arch systems without paru or yay installed

@merdely merdely marked this pull request as draft March 3, 2026 21:02
@merdely
Copy link
Contributor Author

merdely commented Mar 3, 2026

I added code that uses sudo properly when using pacman. In order to not duplicate the archBasedPMSettings structure, I turned it into a function that returns the structure and accepts an argument for requiresSudo

@merdely merdely marked this pull request as ready for review March 3, 2026 21:11
@Purian23
Copy link
Collaborator

Purian23 commented Mar 3, 2026

Hi there, this entire system updater widget is set to get an overhaul in v1.5. Everything should be addressed at that time.

Pacman should go to the end so it's like paru || yay || pacman so the code can still iterate the AUR helpers in order. If they are picked up, they tend to update the pacman system apps as well.

@merdely
Copy link
Contributor Author

merdely commented Mar 3, 2026

I made your suggested change. Though, should we just close this if the overhaul is coming soon?

Copy link
Collaborator

@Purian23 Purian23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We just need to remove the duplicate and this will suffice until the overhaul.

@merdely
Copy link
Contributor Author

merdely commented Mar 3, 2026

I fixed the error. Sorry about that

@merdely merdely requested a review from Purian23 March 3, 2026 22:12
@Purian23 Purian23 merged commit e04c919 into AvengeMedia:master Mar 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants